Skip to content

Conversation

@etobella
Copy link
Member

@etobella etobella commented Oct 15, 2025

Done the history rewritting for not losing the commits from edi_oca and map them in edi_core_oca (Since the split, obviously).

I added JS tests 😉

I avoided the use of odoo_test_helper

simahawk and others added 30 commits October 15, 2025 15:31
* auto btns on/off flag per exchange type
* offer hook to customize how btns are rendered
* improve UI
Raise specific error when exchange type is not found.
To be used to track references on exchanged related records.
Handy method to validate core actions of the edi framework.
Kewyord params were not propagated.
Propagate custom kw args to event handlers.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: edi-15.0/edi-15.0-edi_oca
Translate-URL: https://translation.odoo-community.org/projects/edi-15-0/edi-15-0-edi_oca/
You can now track the original exchange record
that originated the document in the 1st place.
Ease finding which type the current one is an ack for.
You can produce or receive more than one ack for a given exchange.
If that's the case, make sure only the newest is considered.
In case exchange_on is the same, use ID to get the latest on top.
Skip 'output_sent' records by default.
Reason: most of the times when a message is sent out
you don't care about its state anymore.
Any subsequent update from outside should come
with a new input record instead.

Yet, if for any reason this feature is required
you can still enable it by passing 'skip_sent=False'.
@simahawk
Copy link
Contributor

simahawk commented Dec 1, 2025

Copy link
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG overall.

So far the file could be edited in any state and w/o any trace of editing.
This change brings:

* the file is frozen if it has been already processed or sent
* if you really need to change it you can force this by assigning the
  group ``group_edi_override_exchange_file_content`` to the user
* updates to the file are now tracked via checksum changes
model.model few lines below will trigger an access error w/o this change.
@etobella etobella force-pushed the 19.0-mig-edi_core_oca branch 2 times, most recently from 0e6d581 to 8dced0b Compare December 1, 2025 18:35
@etobella
Copy link
Member Author

@simahawk WDYT?

@etobella
Copy link
Member Author

etobella commented Feb 2, 2026

@simahawk is it ok to you to proceed with merge?

simahawk and others added 3 commits February 2, 2026 15:33
There were 3 ways to access the related record.
Now there's only 2 but we still display the warning if the relation with the main record is broken.
@etobella etobella force-pushed the 19.0-mig-edi_core_oca branch from 8dced0b to efe076e Compare February 2, 2026 14:33
Copy link
Contributor

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@etobella
Copy link
Member Author

etobella commented Feb 2, 2026

/ocabot migration edi_core_oca

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Feb 2, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Feb 2, 2026
18 tasks
@etobella
Copy link
Member Author

etobella commented Feb 2, 2026

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-215-by-etobella-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at d22e572. Thanks a lot for contributing to OCA. ❤️

@OCA-git-bot OCA-git-bot merged commit 82d82fb into OCA:19.0 Feb 2, 2026
7 checks passed
@etobella etobella deleted the 19.0-mig-edi_core_oca branch February 2, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.